Skip to content

Define migration process and perform the first migration for second to ms #207

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 9 commits into from
Apr 10, 2025
Merged

Define migration process and perform the first migration for second to ms #207

merged 9 commits into from
Apr 10, 2025

Conversation

v9n
Copy link
Member

@v9n v9n commented Apr 10, 2025 •
edited
Loading

This PR implements:

  • Auto format code on push and make commit back. No need to run any formatter locally
  • Define a migration framework. The migration will perform a backup before run. Backup can be taken through repl.
  • Define our first ever migration that convert timestamp in second to ms.

Note that BadgerDB is rsync-friendly and we will have a different backup with rsync on cron on our aggregator use rsync

@v9n v9n force-pushed the define-migration-process-and-perform-the-first-migration-for-second-to-ms branch from 10646f6 to 0dd13c5 Compare April 10, 2025 10:17
@v9n v9n requested a review from chrisli30 April 10, 2025 17:02
chrisli30
chrisli30 approved these changes Apr 10, 2025
View reviewed changes
Copy link
Member

@chrisli30 chrisli30 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@v9n v9n merged commit 3feb2f1 into staging Apr 10, 2025
14 checks passed
chrisli30 pushed a commit that referenced this pull request Apr 21, 2025
@v9n @devin-ai-integration @chrisli30
Define migration process and perform the first migration for second t…
f3eb84e 
…o ms (#207)

* Add backup ability to run with repl console

* add a migration framework, create second -> ms converted

* autoformat code on push

* Use go1.24 in Dockerfile

* fix new grpc format

* add new packages to test

* add new pkg to test run

* fix AI slop

* fix timestamp change

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@chrisli30 chrisli30 deleted the define-migration-process-and-perform-the-first-migration-for-second-to-ms branch May 14, 2025 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chrisli30 chrisli30 chrisli30 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@v9n @chrisli30